Skip to content

chore: add smoke tests#208

Merged
kptdobe merged 8 commits into
mainfrom
itsmoke
Dec 2, 2025
Merged

chore: add smoke tests#208
kptdobe merged 8 commits into
mainfrom
itsmoke

Conversation

@kptdobe

@kptdobe kptdobe commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Still not perfect but this would have captured the regression introduced by fc9eedf.

Basically:

  • starts a local s3 server
  • starts the da-admin worker locally that points to the s3 server
  • run fetch requests against the da-admin worker

@kptdobe
kptdobe requested a review from tripodsan December 2, 2025 10:16
@tripodsan
tripodsan self-requested a review December 2, 2025 10:30

@tripodsan tripodsan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@codecov

codecov Bot commented Dec 2, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/storage/utils/config.js 71.42% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@kptdobe
kptdobe requested a review from tripodsan December 2, 2025 11:00
@kptdobe

kptdobe commented Dec 2, 2025

Copy link
Copy Markdown
Contributor Author

Somehow wrangler starts a lot of processes. Brute force killing all of them fixes the issue.

@kptdobe
kptdobe merged commit d37157c into main Dec 2, 2025
4 of 5 checks passed
@kptdobe
kptdobe deleted the itsmoke branch December 2, 2025 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants